@sshekhar563 Please check the code style errors reported in the CI checks (or see here https://github.com/joomla/joomla-cms/actions/runs/19208200451/job/54906507158?pr=46428 ). Thanks in advance.

Thanks for the heads-up, @richard67
I’ve fixed all reported code style issues and pushed the updated changes.
The php-cs-fixer and phpcs checks should pass now.

Thanks for the heads-up, @richard67 I’ve fixed all reported code style issues and pushed the updated changes. The php-cs-fixer and phpcs checks should pass now.

@sshekhar563 Code style seems ok now, but PHPstan reports issues. You can see them when checking your changes on GitHub here: https://github.com/joomla/joomla-cms/pull/46428/files and scroll down to the libraries/src/User/CachingUserFactory.phpfile. Please check if you can fix that.

Besides this I see that you place constructor methods before class variables. That's not right, class variables should come first, then the constructor and then other methods. The code style checker might not complain about that, but if you check our other sources you will see it is like that.

Please look at the other files in these folders and update these new files with the same information in the document headers

@sshekhar563 Other thing: Could you rebase this PR to the 6.1-dev branch?

We have discussed it in the maintainers Team.

The referred issue #46369 is not an actual issue, it is a reminder that we will have an issue when we will remove the getInstance() methods in a future release, which can be only a major version due to semantic versioning because it would be a b/c break, so it can happen earliest with 7.0, maybe more likely 8.0.

So it makes sense to introduce a replacement before that like your PR here does.

But as that replacement would be a new feature, it cannot be done with a patch release (also due to semantic versioning), so not with 5.4 or 6.0. That's why it should be rebased to (or re-done for) the 6.1-dev branch.

Let me know if you need help with that.

Thanks for your understanding.

@sshekhar563 did you use AI code assistant?
Please review the code to fit it in to Joomla code style. There still many issues with the generated code.

Thanks for the feedback.
I’ve manually reviewed all three files and updated them to follow Joomla’s core coding style exactly, using the same headers, method order, and formatting as other files in libraries/src/User, Form, and Service.
I also re-ran php-cs-fixer and phpcs, and both now pass cleanly.
Please let me know if you’d like any further adjustments.